Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Ftrack Multiple notes as server action #1794

Merged
merged 4 commits into from
Jul 7, 2021

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Jul 7, 2021

Changes

  • created server version of multiple notes action
OpenPype 2 PRs
OpenPype #1795

return True


def register(session, plugins_presets={}):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not use mutable data structures for argument defaults. They are created during function definition time. All calls to the function reuse this one instance of that data structure, persisting changes between them.


splitter = {
'type': 'label',
'value': '{}'.format(200*"-")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing whitespace around arithmetic operator

@iLLiCiTiT iLLiCiTiT added backend type: enhancement Enhancements to existing functionality module: Ftrack labels Jul 7, 2021
@iLLiCiTiT iLLiCiTiT self-assigned this Jul 7, 2021
@iLLiCiTiT iLLiCiTiT requested a review from 64qam July 7, 2021 15:40
@iLLiCiTiT iLLiCiTiT merged commit 659a8e4 into 2.x/develop Jul 7, 2021
@mkolar mkolar deleted the feature/multiple_notes_server_action branch August 10, 2021 16:02
kalisp added a commit that referenced this pull request Sep 2, 2021
kalisp added a commit that referenced this pull request Sep 2, 2021
kalisp added a commit that referenced this pull request Sep 2, 2021
kalisp added a commit that referenced this pull request Sep 6, 2021
Implemented comparing published files with expected
Changed approach to userSetup
kalisp added a commit that referenced this pull request Sep 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
module: Ftrack type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants